Skip to content

Add '線上聲請平台' endpoint to the sidebar, based on 建班評公字第08023000001號.#300

Merged
RobotHanzo merged 4 commits intoCKStudentCouncil:mainfrom
mm-news:main
Mar 11, 2026
Merged

Add '線上聲請平台' endpoint to the sidebar, based on 建班評公字第08023000001號.#300
RobotHanzo merged 4 commits intoCKStudentCouncil:mainfrom
mm-news:main

Conversation

@mm-news
Copy link
Contributor

@mm-news mm-news commented Mar 10, 2026

This pull request adds a new external link to the sidebar navigation in HeaderSidebar.vue, allowing users to access the online application platform directly from the menu.

Navigation update:

  • Added a new menu item labeled "線上聲請平台" with an external link to the online application platform, including an icon and marking it as not requiring authentication.

@mm-news mm-news marked this pull request as ready for review March 10, 2026 10:12
@RobotHanzo RobotHanzo requested a review from Copilot March 10, 2026 13:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the sidebar navigation in HeaderSidebar.vue to include a new menu item linking users to the external “線上聲請平台”.

Changes:

  • Added a new sidebar endpoint labeled “線上聲請平台” pointing to an external Notion URL.
  • Marked the new endpoint as external: true and requireAuth: false.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +92 to +95
{ name: '編輯法令', url: '/manage/legislation', icon: 'edit', requireAuth: true },
{ name: '編輯公文', url: '/manage/document', icon: 'draw', requireAuth: true },
{ name: '評委文書', url: '/document/judicial', icon: 'balance', requireAuth: false },
{ name: '線上聲請平台', url: 'https://cksc-jc.notion.site/38a4465929c78260b6d681378aba8efc ', icon: 'launch', external: true, requireAuth: false },
Copy link

Copilot AI Mar 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

external: true is added on the endpoint, but the template always renders <q-item :to="endpoint.url">. In Quasar/Vue Router, to is for internal routes; passing an absolute https URL is likely to be treated as a router path and won’t open the external site. Please update the rendering logic to use href/target (and ideally rel="noopener noreferrer") for endpoint.external items, while keeping to for internal routes.

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot open a new pull request to apply changes based on this feedback

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mm-news mm-news marked this pull request as draft March 11, 2026 07:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mm-news mm-news marked this pull request as ready for review March 11, 2026 08:54
@RobotHanzo RobotHanzo merged commit 8f5b205 into CKStudentCouncil:main Mar 11, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants